Skip to content

Conversation

@yiyuan-he
Copy link
Contributor

@yiyuan-he yiyuan-he commented Dec 17, 2024

Description of changes:
Small changes in our contract test assertions to align with recent Gen AI changes in upstream patch.

Relevant changes from upstream patch:

Test plan:
Ran the updated contract tests.
Screenshot 2024-12-17 at 9 30 15 AM

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@yiyuan-he yiyuan-he requested a review from a team as a code owner December 17, 2024 03:20
@yiyuan-he yiyuan-he changed the title update genai contract tests [draft] Update genai contract tests Dec 17, 2024
@mxiamxia mxiamxia merged commit 8796084 into aws-observability:main Jan 7, 2025
4 checks passed
mxiamxia pushed a commit that referenced this pull request Jan 7, 2025
*Description of changes:*
Same as
#976
but merging to our v1.33.x patch branch

*Test plan:*
Ran updated contract tests.
<img width="1920" alt="Screenshot 2025-01-06 at 10 36 24 AM"
src="https://github.com/user-attachments/assets/f8c0fcbc-1b69-4420-ab66-ca426d5c355c"
/>

By submitting this pull request, I confirm that my contribution is made
under the terms of the Apache 2.0 license.
@yiyuan-he yiyuan-he deleted the update-genai-contract-tests branch March 5, 2025 18:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants